- burada "preferencesDialog.setDefaultValues(username,password,port);" username,password ve port gecmek yerine direkt "preferences" objesini parametre olarak gecebilirdin. bu sayede ilerde herhangi bir paremetre degisikliginde sadece "setDefaultValues" methodunu degistirmen yeterli iken senin yazdigin kodda "MainFrame" üzerinde de degisiklik yapman gerekecek.
- Controller da FormEvent "fe" den degerleri alip mesela buradaki gibi "int ageCatId = fe.getAgeId();" bir degiskene ataman gereksiz olmus. switch icinde "switch (ageCatId)" yerine "switch (fe.getAgeId())" kullanabilirdin.
- swing framework de iyi degilim ama "// First Row", "// Next Row" ... kisimlarinin bir method icinde parametrik yapilabilir diye düsünüyorum. kod tekrari cok gibi geldi.
- MainFrame ve FormPanel constructorlar cok uzun geldi, her bir kod blogunu bir method icine atarsan okunmasi kolay olur, kodlar tekrar kullanilabilir olur. ayrica o kod blogunun ne yaptigini anlamak icin methodun ismine bakmak yeterli olur.
Isin gercegi ben de yazmis olsaydim senin gibi yazardim ya da sana benzer hatalar yapardim. Bir insan kod yazarken nerede hata yaptigini anlayamayabiliyor, bu isin dogasinda var. Kusursuz kod yoktur, herkes yazarken bir yerinde elestirilecek bir acik birakir. Ben de simdiki is yerimde yazdigim kodlari diger gelistiricilere "Code Review" icin incelettigimde bircok hata bulduklarini görüyorum. Programlama biliyoruz ama kaliteli kod yazmaki icin ögrenme sürecindeyiz ve bu sürec hic bitmeyecek sanirim. O nedenle sana su kitabi önerebilirim, ben de bu kitabi okuyorum ve her seferinde önceki yazdigim kodlarin hatalarini yüzüme vuruyor.
www.amazon.comAyrica yazdigin koda baktigimda swing ile ilgili bazi noktalari ögrenmis oldum, paylastigin icin tesekkürler.